Skip to content

Feature/load project#113

Closed
12345jy wants to merge 3 commits intothoth-tech:mainfrom
12345jy:feature/load-project
Closed

Feature/load project#113
12345jy wants to merge 3 commits intothoth-tech:mainfrom
12345jy:feature/load-project

Conversation

@12345jy
Copy link
Contributor

@12345jy 12345jy commented Jan 12, 2025

Description

This PR introduces a new "Load Project" feature, including UI changes and backend logic.
Changes were made to four files to implement this feature:

  1. projectLoadUI.js:

    • Added two functions:
      • loadUserProjects(): Handles the logic for loading user projects.
      • ShowProjectLoader(title, getChoices): Displays a modal for selecting a project.
  2. editorMain.js:

    • Updated setupIDEButtonEvents() to add:
      • A new "Load Project" button and its associated event handler.
  3. IDEStartupMain.js:

    • Added LoadUserProjectQueue to manage project loading operations during IDE startup.
  4. index.html:

    • Added the following snippet to integrate the "Load Project" button into the UI:
      <li><button id="LoadProject">Load Project</button></li>

However, clicking the "Load Project" button or the "Load Demo" button does not yet trigger the expected behavior. The changes have been marked with !!! in the comments to help reviewers quickly locate the relevant sections. Further debugging and review are required to fix this issue.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as
    expected)
  • Documentation (update or new)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can
reproduce. Please also list any relevant details for your test configuration

Testing Checklist

  • Tested in latest Chrome
  • Tested in latest Safari
  • Tested in latest Firefox

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have requested a review from ... on the Pull Request

@github-actions
Copy link

github-actions bot commented Jan 12, 2025

🐋 PR Preview!
The preview is no more!
Congrats if this was merged! 😄

@WhyPenguins
Copy link
Contributor

This was a good start! But since it isn't complete I'll close this PR for now 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants